Skip to content

Move Unit Tests for groupby.rolling.corr #54504

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 9 commits into from
Aug 11, 2023

Conversation

omar-elbaz
Copy link
Contributor

Moves recently created unit tests for groupby.rolling.corr to pandas/tests/window/test_groupby.py

Resolves comment in #54499

@omar-elbaz
Copy link
Contributor Author

pre-commit.ci autofix

@mroeschke mroeschke added the Testing pandas testing functions or related to the test suite label Aug 11, 2023
@mroeschke mroeschke added this to the 2.1 milestone Aug 11, 2023
@mroeschke mroeschke merged commit 6222789 into pandas-dev:main Aug 11, 2023
@mroeschke
Copy link
Member

Thanks @omar-elbaz

@lithomas1
Copy link
Member

@meeseeksdev backport 2.1.x

meeseeksmachine pushed a commit to meeseeksmachine/pandas that referenced this pull request Aug 12, 2023
phofl pushed a commit that referenced this pull request Aug 12, 2023
…ng.corr) (#54513)

Backport PR #54504: Move Unit Tests for groupby.rolling.corr

Co-authored-by: omar-elbaz <[email protected]>
@omar-elbaz omar-elbaz deleted the move_unit_test branch August 17, 2023 22:41
mroeschke pushed a commit to mroeschke/pandas that referenced this pull request Aug 18, 2023
* groupby tests

* revert commit

* reverting

* reverting again

* move unit tests

* move test to tests/window

* move UTs

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: Omar Elbaz <[email protected]>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Testing pandas testing functions or related to the test suite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants